Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adding assertions for AddChange test scenario for ResultV2 Test #686

Closed
wants to merge 2 commits into from

Conversation

zhaque44
Copy link
Contributor

@zhaque44 zhaque44 commented May 22, 2024

The updated test accounts for much needed assertions for AddChange scenario in Result V2 test:

This code snippet initializes an expectedFileChanges variable, which is containing the expected changes. Each file is associated with a set of object changes represented by ObjectChanges. Within each file's changes, specific objects are associated with an array of Change structs, where each struct represents a change in values. The g.Expect(result.FileChanges) statement asserts that the FileChanges field of the result variable matches the expected changes defined.

@zhaque44
Copy link
Contributor Author

zhaque44 commented May 22, 2024

Hi @souleb hope you are doing well, I made the necessary changes recommended here: #682 (comment) please review this when you have a minute

@zhaque44
Copy link
Contributor Author

@souleb hey can you take a look at these changes please

@zhaque44 zhaque44 closed this Jun 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants